Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] theme_avantgarde: s_carousel text contrast #1021

Conversation

mano-odoo
Copy link
Contributor

The o_cc3 is rendering a white color depending on the the palette. Using o_cc5 ensure a white text.

task-4381722

@robodoo
Copy link
Collaborator

robodoo commented Dec 4, 2024

Pull request status dashboard

@mano-odoo mano-odoo marked this pull request as ready for review December 4, 2024 16:36
@mano-odoo
Copy link
Contributor Author

@odoo/rd-website

Copy link

@chrisdt1998 chrisdt1998 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qsm-odoo
Copy link
Contributor

@robodoo delegate=chrisdt1998
@chrisdt1998

Although, is that really a thing to do in stable? 🤔

@chrisdt1998
Copy link

@mano-odoo after discussing with QSM, please could you target this branch to master? Generally we avoid changing XML in stable and in this case, this change isn't critical (the user can just manually change the text color) so it's sufficient to just do the change on master IMO.

Thanks :)

The `o_cc3` is rendering a white color depending on the the palette.
Using `o_cc5` ensure a white text on most palettes.

task-4381722
@mano-odoo mano-odoo force-pushed the 18.0-avantgarde-s_carousel-contrast-mano branch from 84cd672 to f40aa7a Compare February 19, 2025 14:02
@mano-odoo mano-odoo changed the base branch from 18.0 to master February 19, 2025 14:02
@mano-odoo
Copy link
Contributor Author

@chrisdt1998 should be good to go (sorry forgot to ping again) Thanks a lot!

@chrisdt1998
Copy link

@robodoo r+

robodoo pushed a commit that referenced this pull request Feb 20, 2025
The `o_cc3` is rendering a white color depending on the the palette.
Using `o_cc5` ensure a white text on most palettes.

task-4381722

closes #1021

Signed-off-by: Christopher du Toit (chto) <[email protected]>
@robodoo robodoo added the 18.3 label Feb 20, 2025
@robodoo robodoo closed this Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants